Skip to content

Conversation

@mulkieran
Copy link
Member

@mulkieran mulkieran commented Oct 28, 2025

Related stratis-storage/project#805

Summary by CodeRabbit

  • Chores
    • Updated CI/CD infrastructure to use Fedora 42 for build and testing processes.
    • Updated documentation build tooling dependencies.

@mulkieran mulkieran self-assigned this Oct 28, 2025
@mulkieran mulkieran moved this to In Review in 2025October Oct 28, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 28, 2025

Walkthrough

The Fedora container image tag in the GitHub Actions workflow was updated from fedora:41 to fedora:42 for three jobs (docs-checks, ci-checks, spelling-checks), and the Copr repository for Zola was changed from atim/zola to pgdev/zola in the docs-checks job. No other workflow logic was modified.

Changes

Cohort / File(s) Change Summary
GitHub workflow updates
​.github/workflows/main.yml
Updated Fedora base image tag from fedora:41 to fedora:42 in jobs docs-checks, ci-checks, and spelling-checks. Changed Copr repository for Zola from atim/zola to pgdev/zola in the docs-checks job.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Files to spot-check: ​.github/workflows/main.yml (verify image tags and Copr repo string).

Poem

🐰 I hopped a minor version, light upon my paws,
Fedora forty-one to forty-two — a gentle cause.
Three little jobs donned fresher coats, a Zola path realigned,
No logic bent, no tangled threads, just tidy things defined.
🥕

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the main change: updating the development environment container image from Fedora 41 to Fedora 42 across GitHub Actions workflows.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6f71ab2 and 96b97d3.

📒 Files selected for processing (1)
  • .github/workflows/main.yml (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: docs-checks
🔇 Additional comments (2)
.github/workflows/main.yml (2)

16-16: Fedora version updates are consistent and well-documented.

The container image tag updates from fedora:41 to fedora:42 across all three jobs (docs-checks, ci-checks, spelling-checks) are consistent and align with the PR objective to update the development environment. The inline comments clearly mark these as the current development environment.

Also applies to: 32-32, 44-44


24-24: Verify Copr repository change for Fedora 42 compatibility.

The Copr repository for Zola was changed from atim/zola to pgdev/zola. This is a significant change that requires verification:

  • Is pgdev/zola the correct/recommended Copr repository for Fedora 42?
  • Does it provide compatible Zola packages for your use case?
  • What is the reason for switching repositories (e.g., atim/zola discontinued, incompatible with F42)?

Please document the rationale for this repository change in the PR description or commit message if not already present.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mulkieran mulkieran moved this from In Review to In Progress in 2025October Oct 28, 2025
@mulkieran mulkieran moved this from In Progress to Pending in 2025October Oct 28, 2025
the Mulhern added 2 commits November 4, 2025 12:39
Signed-off-by: the Mulhern <amulhern@amulhern.bos.csb>
Signed-off-by: the Mulhern <amulhern@amulhern.bos.csb>
@mulkieran mulkieran merged commit 81d2974 into stratis-storage:master Nov 4, 2025
5 checks passed
@mulkieran mulkieran deleted the issue_project_805 branch November 4, 2025 17:40
@github-project-automation github-project-automation bot moved this from Pending to Done in 2025October Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant